Skip to content

xfstests: get XFSTESTS_BLACKLIST from file#25023

Open
lansuse wants to merge 1 commit intoos-autoinst:masterfrom
lansuse:master
Open

xfstests: get XFSTESTS_BLACKLIST from file#25023
lansuse wants to merge 1 commit intoos-autoinst:masterfrom
lansuse:master

Conversation

@lansuse
Copy link
Contributor

@lansuse lansuse commented Mar 19, 2026

set known issue file url to XFSTESTS_BLACKLIST if it starts with http:// or https://
For instance: XFSTESTS_BLACKLIST=http://example.com/blacklist.yaml

@github-actions
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@lansuse lansuse force-pushed the master branch 3 times, most recently from cb787e8 to 4942bf6 Compare March 19, 2026 15:34
set known issue file url to XFSTESTS_BLACKLIST if it starts with http:// or https://
For instance: XFSTESTS_BLACKLIST=http://example.com/blacklist.yaml
@lansuse lansuse marked this pull request as ready for review March 20, 2026 05:16
@czerw
Copy link
Contributor

czerw commented Mar 20, 2026

Why do we need YAML support in XFSTESTS_BLACKLIST? We already have XFSTESTS_KNOWN_ISSUES. Can we use it? I check our jobs and I see:

Can we use just one variable ? Bot variables points into same repo anyway (just different files for different product).

@lansuse
Copy link
Contributor Author

lansuse commented Mar 20, 2026

Why do we need YAML support in XFSTESTS_BLACKLIST? We already have XFSTESTS_KNOWN_ISSUES. Can we use it? I check our jobs and I see:

* maintenance https://openqa.suse.de/tests/21415724#settings: there is `XFSTESTS_KNOWN_ISSUES` and there is no blacklist

* development https://openqa.suse.de/tests/21340875#settings: there is `XFSTESTS_BLACKLIST` and there are no known issues

Can we use just one variable ? Bot variables points into same repo anyway (just different files for different product).

In a brief, we just extend original XFSTESTS_BLACKLIST to use a yaml file. That's for change history tracking, and more readable. This won't impact maintenance testing.
Beside that, we can shrink our existing yaml file with XFSTESTS_BLACKLIST format. (It can delete more than 10k lines.)
And it cannot use one variable as XFSTESTS_KNOWN_ISSUES used LTP::Whitelist.

More details in, #24639

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants